]> dgit.raspbian.org Git - gnome-shell.git/commitdiff
appDisplay: Use hookup_style() to bind app-view-controls spacing
authorFlorian Müllner <fmuellner@gnome.org>
Tue, 9 Jul 2013 21:15:02 +0000 (23:15 +0200)
committerFlorian Müllner <fmuellner@gnome.org>
Wed, 24 Jul 2013 13:22:05 +0000 (15:22 +0200)
With the monkey-patched ClutterBoxLayout, we no longer need this
code to hook up the 'spacing' property to CSS.

https://bugzilla.gnome.org/show_bug.cgi?id=703905

js/ui/appDisplay.js

index 144419cec6ec491a49712054d068dc6af8d41e89..9132d18b9047df208e623f8cc5319d29e3979fdb 100644 (file)
@@ -358,17 +358,6 @@ const ControlsBoxLayout = Lang.Class({
         let totalSpacing = this.spacing * (childrenCount - 1);
         return [maxMinWidth * childrenCount + totalSpacing,
                 maxNaturalWidth * childrenCount + totalSpacing];
-    },
-
-    vfunc_set_container: function(container) {
-        if(this._styleChangedId) {
-            this._container.disconnect(this._styleChangedId);
-            this._styleChangedId = 0;
-        }
-        if(container != null)
-            this._styleChangedId = container.connect('style-changed', Lang.bind(this,
-                    function() { this.spacing = this._container.get_theme_node().get_length('spacing'); }));
-        this._container = container;
     }
 });
 
@@ -416,8 +405,9 @@ const AppDisplay = new Lang.Class({
         this.actor.add(this._viewStack, { expand: true });
 
         let layout = new ControlsBoxLayout({ homogeneous: true });
-        this._controls = new St.Widget({ style_class: 'app-view-controls' });
-        this._controls.set_layout_manager(layout);
+        this._controls = new St.Widget({ style_class: 'app-view-controls',
+                                         layout_manager: layout });
+        layout.hookup_style(this._controls);
         this.actor.add(new St.Bin({ child: this._controls }));